cleanup: Don't assume hasOwnProperty() method on objects

Since ES5, it is possible to create objects with no prototype at all:

    let foo = Object.create(null);

Those object won't have any builtin properties like hasOwnProperty(),
which is why eslint added a corresponding rule to its default rule set.

While this isn't an issue that affects our code, there's no harm in fol-
lowing the recommendation and call the method through Object.prototype.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/716

diff --git a/js/ui/layout.js b/js/ui/layout.js
index 85bbd24c22abbf9372fcc6d6946cf833939af092..bfe19bfcb58029604bdb569eaf12ca2afeba31a1 100644 (file)
--- a/js/ui/layout.js
+++ b/js/ui/layout.js
@@ -861,7 +861,7 @@ var LayoutManager = GObject.registerClass({
         // We can't use Params.parse here because we want to drop
         // the extra values like ancestorData.actor
         for (let prop in defaultParams) {
-            if (!params.hasOwnProperty(prop))
+            if (!Object.prototype.hasOwnProperty.call(params, prop))
                 params[prop] = ancestorData[prop];
         }
 

diff --git a/js/ui/sessionMode.js b/js/ui/sessionMode.js
index 583780111a364a0811a80faf4e758ae66951209a..55a422f3d5a2899b96350b0d58438fa236b8702b 100644 (file)
--- a/js/ui/sessionMode.js
+++ b/js/ui/sessionMode.js
@@ -111,7 +111,7 @@ function _loadMode(file, info) {
     let suffix = name.indexOf('.json');
     let modeName = suffix == -1 ? name : name.slice(name, suffix);
 
-    if (_modes.hasOwnProperty(modeName))
+    if (Object.prototype.hasOwnProperty.call(_modes, modeName))
         return;
 
     let fileContent, success_, newMode;